Skip to content

fix(passport): update access token expiry check#2657

Merged
CodeSchwert merged 8 commits into
mainfrom
ID-3828/update-access-token-expiry
Jul 18, 2025
Merged

fix(passport): update access token expiry check#2657
CodeSchwert merged 8 commits into
mainfrom
ID-3828/update-access-token-expiry

Conversation

@CodeSchwert
Copy link
Copy Markdown
Contributor

@CodeSchwert CodeSchwert commented Jul 4, 2025

Hi👋, please ensure the PR title follows the below standards:

  • PR is titled with conventional commit style naming: type(scope): message. For example: feat(passport): my new feature
  • If you have introduced modification that necessitates immediate adjustments by this SDK's users to their applications, clients, or integrations to avert disruptions to existing features or functionalities, add a ! after the type(scope), for example feat(passport)!: my new breaking feature

Summary

ID-3828

  • Force token refresh if access token is going to expire (within 30 secs).

Detail and impact of the change

Added

Changed

Deprecated

Removed

Fixed

Security

Anything else worth calling out?

@CodeSchwert CodeSchwert requested a review from a team as a code owner July 4, 2025 08:24
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jul 4, 2025

View your CI Pipeline Execution ↗ for commit 9ad2e5e

Command Status Duration Result
nx affected -t build,lint,test ✅ Succeeded 2m 57s View ↗
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 1m 17s View ↗

☁️ Nx Cloud last updated this comment at 2025-07-18 01:37:54 UTC

nattb8
nattb8 previously approved these changes Jul 6, 2025
Comment thread packages/passport/sdk/src/utils/token.ts Outdated
@CodeSchwert CodeSchwert requested a review from haydenfowler July 14, 2025 00:48
Comment thread packages/passport/sdk/src/utils/token.ts Outdated
Comment thread packages/passport/sdk/src/utils/token.ts Outdated
Comment thread packages/passport/sdk/src/utils/token.ts Outdated
@CodeSchwert CodeSchwert added this pull request to the merge queue Jul 18, 2025
Merged via the queue into main with commit 219490c Jul 18, 2025
8 checks passed
@CodeSchwert CodeSchwert deleted the ID-3828/update-access-token-expiry branch July 18, 2025 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants